Skip to content

Aws t26 s3 #21

Merged
merged 49 commits into from
Oct 1, 2020
Merged

Aws t26 s3 #21

merged 49 commits into from
Oct 1, 2020

Conversation

ashle001
Copy link
Contributor

@ashle001 ashle001 commented Oct 1, 2020

Please merge into master. Code needs formatting and may want to add IP address restriction policy to bucket after testing but the currently included components are working as designed

Copy link
Contributor

@badra001 badra001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove entirely the locals.tf file, it is not needed in the module

Copy link
Contributor

@badra001 badra001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@badra001 badra001 merged commit b3906b1 into master Oct 1, 2020
@badra001 badra001 deleted the aws-t26-s3 branch October 1, 2020 19:24
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants